Skip to content

Conversation

elupus
Copy link

@elupus elupus commented Jun 23, 2025

  • Add support for building p-net from the evaluation binaries and samples provided on our releases.
  • Rework src based build to build from a locally provided src file instead of github repository.

This allow us to build p-net from releases instead
of source code which i useful for evaluation.
@elupus elupus force-pushed the rtljopl/binary_build branch from e2c610b to 19331dc Compare June 24, 2025 08:44
@elupus elupus requested review from amn-rtl and pyhys June 24, 2025 11:51
@elupus elupus marked this pull request as ready for review June 24, 2025 11:51
@elupus elupus force-pushed the rtljopl/binary_build branch from 6500216 to 419d7f2 Compare June 27, 2025 06:03
Copy link

@amn-rtl amn-rtl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some comments

@@ -0,0 +1,4 @@
# P-Net Yocto Files

Place source code of p-net in `p-net-sources.zip` here
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update the zip file name to correspond to the recipe.

LIC_FILES_CHKSUM = "file://CMakeLists.txt;beginline=1;endline=14;md5=aa7e6f963fad674b02c35dc8e9aa5519"
SECTION = "console/network"

DEPENDS = "p-net-eval"
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where is "p-net-eval" defined?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm. Det behöver ju vara pnet-evaluation.


install -d ${D}${prefix}/share/profinet
install -m 0755 ${S}/share/profinet/* ${D}${prefix}/share/profinet
}
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is the intention to install a static library in the image? If so, does this actually work?
(My experience is that a "-staticdev" package is necessary to install a static library.)

Copy link
Author

@elupus elupus Oct 13, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No this is not the shared libraries, this are the network helper scripts.

@CLAassistant
Copy link

CLAassistant commented Sep 23, 2025

CLA assistant check
All committers have signed the CLA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants